Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] Streamable HTTPCallable functions (Alt. to #14290) #14376

Closed
wants to merge 47 commits into from

Conversation

ncooke3
Copy link
Member

@ncooke3 ncooke3 commented Jan 23, 2025

No description provided.

eBlender and others added 30 commits December 19, 2024 14:32
Add initial support for Streamable functions.
Change call to AsyncThrowingStream<HTTPSCallableResult, Error>
testing check.sh
Remove old test function.
Remove old test
Add a full working test for stremableFunction. Refactor.
Add Json capabilities to parse an HTTP result back and forth.

Updating Unit tests.
Add doc for processResponseDataForStreamableContent.
Update func to have a callback.
Update UnitTests
Remove spaces.
Fetch Main branch.
Project clean up.
Updated to renamed code. From callableResultFromResponse to callableResult
Run style.sh
Fix concurrency " mutation of captured var 'response' in concurrently-executing code" and typos.
This file is not needed.
This file is not needed.
Add throws
Remove DO - CATCH
@google-oss-bot
Copy link

1 Warning
⚠️ Did you forget to add a changelog entry? (Add #no-changelog to the PR description to silence this warning.)

Generated by 🚫 Danger

Copy link
Contributor

github-actions bot commented Jan 23, 2025

Apple API Diff Report

Commit: ab2be0b
Last updated: Mon Jan 27 09:07 PST 2025
View workflow logs & download artifacts


FirebaseFunctions

Structures

Callable
[ADDED] stream(_:)
Swift:
+  public func stream ( _ data : Request ) async throws -> AsyncThrowingStream < Response , Error >

@ncooke3 ncooke3 closed this Jan 29, 2025
@ncooke3 ncooke3 deleted the iOS-Stremable-Functions branch January 29, 2025 15:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants